Merged
Conversation
- Introduced mandatory pre-workflow checks for existing sessions to prevent conflicts and enable session resumption. - Updated workflow entry process to ensure session checks are performed before initiating new workflows. - Improved session ID management with detailed usage patterns and best practices for handling multiple workflows. - Enhanced enforcement rules to prioritize session awareness and conflict resolution. - Adjusted output formatting for session information to display full session IDs for clarity.
- Updated the transition failure message to specify that past user approvals should not be considered current, enhancing clarity on user approval requirements for workflow transitions.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces session management enhancements and updates across multiple files to improve workflow handling, enforce session awareness, and refine user prompts. The changes emphasize mandatory session checks, conflict resolution, and best practices for managing workflows.
Session Management Enhancements:
.cursor/rules/execute-tasks.mdc: Added mandatory session awareness steps, including session discovery, applicability assessment, and conflict resolution, before initiating workflows. Updated enforcement rules to prioritize session checks over workflow discovery. [1] [2]src/accordo_cli/utils/bootstrap.py: Integrated session management commands (workflow_cache_management,workflow_state) and updated best practices to ensure session continuity, isolation, and tracking. Introduced detailed enforcement rules for session handling. [1] [2] [3] [4] [5] [6]User Prompt Refinements:
src/accordo_workflow_mcp/prompts/phase_prompts.py: Modified the error message in_handle_dynamic_workflowto explicitly disallow considering past user approvals as current ones.src/accordo_workflow_mcp/prompts/phase_prompts.py: Updated_handle_cache_list_operationto display full session IDs instead of truncated ones, improving clarity in session listings.